Skip to content

pre-commit: automatically push fixes to branch#9027

Merged
alexreinking merged 13 commits intomainfrom
alexreinking/pre-commit/automatic
Mar 15, 2026
Merged

pre-commit: automatically push fixes to branch#9027
alexreinking merged 13 commits intomainfrom
alexreinking/pre-commit/automatic

Conversation

@alexreinking
Copy link
Copy Markdown
Member

To reduce new-contributor overhead @abadams suggested it would be nice to automatically fix up issues detected by the pre-commit hooks. This PR implements that feature.

I will test here by pushing invalid commits.

@alexreinking alexreinking added the skip_buildbots Do not run buildbots on this PR. Must add before opening PR as we scan labels immediately. label Mar 14, 2026
Copy link
Copy Markdown
Contributor

@mcourteaux mcourteaux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool stuff! This will save us a lot of time, running clang-format for us.

@alexreinking alexreinking merged commit 6117710 into main Mar 15, 2026
1 check passed
@alexreinking alexreinking deleted the alexreinking/pre-commit/automatic branch March 16, 2026 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip_buildbots Do not run buildbots on this PR. Must add before opening PR as we scan labels immediately.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants